Skip to content

Conversation

@jaz1-nordic
Copy link
Contributor

@jaz1-nordic jaz1-nordic commented Aug 21, 2024

mbox_error_cases tests must be run with the nordic-flpr snippet to prevent bus fault error while accessing VPR registers.

Upstream PR: zephyrproject-rtos/zephyr#75114

@nordic-segl
Copy link
Contributor

nordic-segl commented Aug 21, 2024

I add this fix here:
zephyrproject-rtos/zephyr#75114

  1. Not sure how to handle commits by other author in my PR - let me know if commit "tests: drivers: mbox: mbox_error_cases: Fix for BUS FAULT on nrf54l15 needs changes.
  2. This PR can be changed to [nrf fromlist]. Don't forget to add
    Upstream PR: NRFX-5977 Add mbox_error_cases zephyrproject-rtos/zephyr#75114
    in the commit body.

@jaz1-nordic jaz1-nordic force-pushed the mbox_error_cases_bugfix branch from e3eeb48 to 5805f90 Compare August 27, 2024 09:19
@jaz1-nordic jaz1-nordic changed the title [nrf noup] tests: drivers: mbox: Fix for BUS FAULT on nrf54l15 [nrf fromlist] tests: drivers: mbox: Fix for BUS FAULT on nrf54l15 Aug 27, 2024
@jaz1-nordic jaz1-nordic force-pushed the mbox_error_cases_bugfix branch 4 times, most recently from 210094b to b0c55f7 Compare September 9, 2024 09:26
@jaz1-nordic jaz1-nordic force-pushed the mbox_error_cases_bugfix branch from b0c55f7 to dff2bb1 Compare September 12, 2024 14:36
rlubos
rlubos previously requested changes Sep 12, 2024
Copy link
Contributor

@rlubos rlubos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be [nrf fromtree] commit, not [nrf noup] as it's cherry-picked from upstream:
zephyrproject-rtos/zephyr@bd93623

mbox_error_cases tests must be run with the nordic-flpr snippet
to prevent bus fault error while accessing VPR registers.

Signed-off-by: Jakub Zymelka <[email protected]>
(cherry picked from commit bd93623)
@jaz1-nordic jaz1-nordic force-pushed the mbox_error_cases_bugfix branch from dff2bb1 to 719b3c6 Compare September 13, 2024 05:54
@rlubos rlubos dismissed their stale review September 13, 2024 07:35

Addressed

@rlubos rlubos merged commit 1ab791d into nrfconnect:main Sep 13, 2024
16 checks passed
@jaz1-nordic jaz1-nordic deleted the mbox_error_cases_bugfix branch September 13, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants